Skip to content

[Fix] Astra provider CI fails when focused tests are omitted - #1530

Draft
zoomote[bot] wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w
Draft

[Fix] Astra provider CI fails when focused tests are omitted#1530
zoomote[bot] wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w

Conversation

@zoomote

@zoomote zoomote Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

​Opened on behalf of @navedmerchant. View the task or mention @roomote for follow-up asks.

What changed

  • Keeps hyphen-suffixed direct tests such as openai-native-usage.spec.ts in changed-code mutation runs instead of dropping them when a dot-suffixed direct test also exists.
  • Adds focused decision-table coverage for Astra reasoning normalization, Responses Lite routing, cache behavior, temperature, token fields, and sequential tool calls across the providers merged in [Feat] Add verified GPT-6 Astra support across providers #1506.
  • Simplifies equivalent cache-total and exact-model predicates so mutation testing measures observable behavior rather than module-initialization artifacts.

Why this change was made

The mutation gate on #1506 omitted an existing focused usage suite and left provider guard branches under-specified. The feature was merged while the final CI remediation was still being validated, so these fixes are isolated in this follow-up.

Impact

Changed-code mutation testing now exercises all directly related focused suites and passes for the Astra provider changes. Runtime behavior from #1506 is preserved.

Related PRs

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Review status

This PR was opened by an automated account. A human maintainer must verify the change intent, provenance, and validation before merging.

Current step: Resolve the merge conflicts. The review sequence resumes after the branch is mergeable.

Review-state labels are managed by this workflow; do not edit them manually.

@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions github-actions Bot added has-conflicts PR has merge conflicts with the base branch and removed has-conflicts PR has merge conflicts with the base branch labels Sep 6, 2026
@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit-review-active

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: eae082df-3ba7-4c65-a52c-69a2272e4869

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-conflicts PR has merge conflicts with the base branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant